Skip to content

Remove per-commit checking from CI workflow#46

Merged
jerrysxie merged 3 commits into
OpenDevicePartnership:mainfrom
jerrysxie:remove-per-commit-checking
May 27, 2026
Merged

Remove per-commit checking from CI workflow#46
jerrysxie merged 3 commits into
OpenDevicePartnership:mainfrom
jerrysxie:remove-per-commit-checking

Conversation

@jerrysxie

Copy link
Copy Markdown
Contributor

Remove the commit_list job and all per-commit matrix strategies from the check workflow. Each CI job now runs once against the PR head or push ref instead of iterating over every individual commit.

This simplifies the workflow and reduces CI resource usage.

Assisted-by: GitHub Copilot:claude-opus-4.6

Remove the commit_list job and all per-commit matrix strategies from
the check workflow. Each CI job now runs once against the PR head or
push ref instead of iterating over every individual commit.

This simplifies the workflow and reduces CI resource usage.

Assisted-by: GitHub Copilot:claude-opus-4.6
@jerrysxie jerrysxie self-assigned this May 27, 2026
@github-project-automation github-project-automation Bot moved this to In progress in ODP v0.2 May 27, 2026
@jerrysxie jerrysxie marked this pull request as ready for review May 27, 2026 16:55
@jerrysxie jerrysxie requested a review from a team as a code owner May 27, 2026 16:55

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the per-commit CI fan-out in the check GitHub Actions workflow so each job runs once per PR head (or push ref), reducing CI load and simplifying the workflow structure.

Changes:

  • Deleted the commit_list job and its commit enumeration logic.
  • Removed per-commit matrix strategies and ref: ${{ matrix.commit }} checkouts across jobs.
  • Simplified job naming to no longer include per-commit identifiers.

Comment thread .github/workflows/check.yml
Comment thread .github/workflows/check.yml
jerrysxie added 2 commits May 27, 2026 11:59
Fix nostd.yml: replace check.yml copy with proper no-std workflow
Update check.yml header comments and remove stale semver comment
@jerrysxie jerrysxie enabled auto-merge (squash) May 27, 2026 17:06
@jerrysxie jerrysxie merged commit f8eb95c into OpenDevicePartnership:main May 27, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in ODP v0.2 May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants